Greptile Summary

This PR refactors internal/common/metrics/scheduler_metrics.go by introducing 16 package-level constants for all Prometheus label name strings (e.g., labelPool, labelCluster, labelQueue) and replacing every raw string literal in the descriptor definitions with the corresponding constant. This improves consistency and reduces the risk of future typo-driven label mismatches.

Key observations:

• All previously flagged incomplete conversions (QueueAllocatedDesc, QueueUsedDesc, QueueLeasedPodCountDesc still using the raw "cluster" string) have been addressed in the current HEAD and all 16 constants are applied consistently across every descriptor.
• The new constants are unexported (lowercase). While this is sufficient for current single-file usage, it means other files in the metrics package cannot reference them without repeating raw string literals — potentially reintroducing the problem the PR aims to prevent.
• A pre-existing issue was noticed (not introduced by this PR): AllDescs contains duplicate entries for MedianQueueDurationDesc and QueuePriorityDesc, and is missing entries for QueueDistinctSchedulingKeysDesc, NodeJobPhaseCounterDesc, and ClusterCordonedStatusDesc. Since this file is being actively modified, it would be a good opportunity to fix these as well.

Confidence Score: 4/5

• Safe to merge — purely a refactor with no runtime behavior changes; all constants map 1:1 to their original string values.
• The PR correctly replaces every string literal with its corresponding constant and addresses all previously flagged issues. No logic changes are made. A minor style concern exists around constant visibility (unexported), and a pre-existing AllDescs hygiene issue is noted but not introduced by this PR.
• No files require special attention for correctness; AllDescs in scheduler_metrics.go has pre-existing duplicate/missing entries worth cleaning up.

Important Files Changed

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[Label Constants Block<br/>labelPool, labelQueue, labelCluster, ...] -->|used in| B[Prometheus Desc Variables<br/>QueueAllocatedDesc, ClusterCapacityDesc, ...]
    B -->|registered via| C[AllDescs slice]
    C -->|iterated in| D[Describe func]
    B -->|referenced by| E[New* metric constructor funcs<br/>e.g. NewQueueAllocated, NewClusterCapacity]
    E -->|emit| F[Prometheus Metrics]

_{Reviews (3): Last reviewed commit: "update missed cluster labels" | Re-trigger Greptile}